Skip to content

[Quality Management] Improve permissions sets and standardize naming, Change the link displayed on the QM setup guide#7416

Merged
JakovljevicDusan merged 11 commits intomainfrom
bugs/QM-ImprovePermissionsSetsAndStandardizeNaming
Mar 30, 2026
Merged

[Quality Management] Improve permissions sets and standardize naming, Change the link displayed on the QM setup guide#7416
JakovljevicDusan merged 11 commits intomainfrom
bugs/QM-ImprovePermissionsSetsAndStandardizeNaming

Conversation

@JakovljevicDusan
Copy link
Copy Markdown
Contributor

@JakovljevicDusan JakovljevicDusan commented Mar 27, 2026

Changes:

  1. Drop permissionsetextension 20400 "D365 BASIC - QltyMgmt" extends "D365 BASIC", and replace with "D365 BASIC ISV".
  2. Rename permissionset 20405 "QltyMngmnt - Edit" to "QltyMgmt - Admin" ('Quality Management - Administrator')
  3. Include HasAdminSupervisorRole() into reopen inspection action check
  4. Limit "permissionset 20404 QltyMgmt - Inspector" to operational-only role
  5. Standardize 'QltyMngmnt' to 'QltyMgmt' across permission sets and related code
image
  1. In Setup Guide, aligning the UX with 28.0 and 28.x to avoid issues, but the UX might be changed later on (only on main).

Fixes AB#626135
Fixes AB#626590
Fixes AB#622404

"D365 BASIC" -> "D365 BASIC ISV"

"QltyMngmnt - Edit" -> "QltyMngmnt - Admin"

Include HasAdminSupervisorRole() into reopen inspection action check

4.	Limit "permissionset 20404 QltyMngmntInspector" to operational-only role

pragmas

Fix pragma

[Quality Management] [28.0] Improve Permissions (#7289)

Changes:
1. Drop permissionsetextension 20400 "D365 BASIC - QltyMngmnt" extends
"D365 BASIC", and replace with "D365 BASIC ISV".
2. Rename permissionset 20405 "QltyMngmnt - Edit" to "QltyMngmnt -
Admin" ('Quality Management - Administrator')
3. Include HasAdminSupervisorRole() into reopen inspection action check
4. Limit "permissionset 20404 QltyMngmntInspector" to operational-only
role

needs to be open and approved. Submitting PRs with no linked issues or
unapproved issues is highly discouraged. -->
Fixes
[AB#622201](https://dynamicssmb2.visualstudio.com/1fcb79e7-ab07-432a-a3c6-6cf5a88ba4a5/_workitems/edit/622201)

[Quality Management] Standardize 'QltyMngmnt' to 'QltyMgmt' across permission sets and related code

Include permissionsetextension
@github-actions github-actions bot added Build: scripts & configs Build scripts and configuration files AL: Apps (W1) Add-on apps for W1 labels Mar 27, 2026
@github-actions github-actions bot added this to the Version 29.0 milestone Mar 27, 2026
Copy link
Copy Markdown

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

AL Documentation Audit

Documentation gaps were detected in the following apps:

  • Quality-Management-Test-Library: 0% documentation coverage
  • Quality-Management-Tests: 0% documentation coverage
  • Quality-Management: 0% documentation coverage

To generate documentation, run /al-docs init or /al-docs update using GitHub Copilot CLI or Claude Code.
This review is for awareness to help keep documentation in sync with code changes. It is okay to dismiss this request.

@JakovljevicDusan JakovljevicDusan marked this pull request as ready for review March 30, 2026 12:37
@JakovljevicDusan JakovljevicDusan requested review from a team as code owners March 30, 2026 12:37
@JakovljevicDusan JakovljevicDusan enabled auto-merge (squash) March 30, 2026 12:37
auto-merge was automatically disabled March 30, 2026 12:48

Pull Request is not mergeable

@JakovljevicDusan JakovljevicDusan merged commit c551b5d into main Mar 30, 2026
49 checks passed
@JakovljevicDusan JakovljevicDusan deleted the bugs/QM-ImprovePermissionsSetsAndStandardizeNaming branch March 30, 2026 12:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

AL: Apps (W1) Add-on apps for W1 Build: scripts & configs Build scripts and configuration files

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants